Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(engine-core): Improve binary search logic #4054

Merged
merged 2 commits into from Oct 28, 2020

Conversation

williamluke4
Copy link
Contributor

@williamluke4 williamluke4 commented Oct 27, 2020

The idea behind this is:

  1. Make the code more readable
  2. If the binary can not be found to display all the folders that it looked in. Not just the
    contents of the last search space (Current)

@timsuchanek
Copy link
Contributor

Let's merge this after the release, as this is critical logic

@timsuchanek
Copy link
Contributor

@williamluke4 can you give some context what the improvements are for?

@timsuchanek timsuchanek added this to the Release 2.11.0 milestone Oct 28, 2020
@williamluke4 williamluke4 self-assigned this Oct 28, 2020
@timsuchanek timsuchanek merged commit d277ff3 into master Oct 28, 2020
@timsuchanek timsuchanek deleted the chore/engine-core/improve-binary-search-logic branch October 28, 2020 15:59
Jolg42 added a commit that referenced this pull request Oct 29, 2020
* master:
  fix(deps): update engines to v2.11.0-7-2034ce0e038ece25b48b5180b41ccb48d2c3344f (#4091)
  chore(deps): update devdependencies (non-major) (#4084)
  fix(deps): update engines to v2.11.0-3-6fafbb461b7e0c140280a7794ed19a4f7035f577 (#4080)
  chore(engine-core): Improve binary search logic (#4054)
  chore(deps): update dependency esbuild to v0.7.22 (#4078)
  fix(deps): update engines to v2.11.0-2-6b72297958d424b0e0a0829a95b76ccc966132fd (#4073)
Jolg42 added a commit that referenced this pull request Oct 29, 2020
* master:
  fix(deps): update engines to v2.11.0-7-2034ce0e038ece25b48b5180b41ccb48d2c3344f (#4091)
  chore(deps): update devdependencies (non-major) (#4084)
  fix(deps): update engines to v2.11.0-3-6fafbb461b7e0c140280a7794ed19a4f7035f577 (#4080)
  chore(engine-core): Improve binary search logic (#4054)
  chore(deps): update dependency esbuild to v0.7.22 (#4078)
  fix(deps): update engines to v2.11.0-2-6b72297958d424b0e0a0829a95b76ccc966132fd (#4073)
Jolg42 added a commit that referenced this pull request Oct 29, 2020
* master:
  fix(deps): update engines to v2.11.0-7-2034ce0e038ece25b48b5180b41ccb48d2c3344f (#4091)
  chore(deps): update devdependencies (non-major) (#4084)
  fix(deps): update engines to v2.11.0-3-6fafbb461b7e0c140280a7794ed19a4f7035f577 (#4080)
  chore(engine-core): Improve binary search logic (#4054)
  chore(deps): update dependency esbuild to v0.7.22 (#4078)
  fix(deps): update engines to v2.11.0-2-6b72297958d424b0e0a0829a95b76ccc966132fd (#4073)
  chore(deps): update devdependencies (non-major) (#4065)
  chore: update engines pkg (#4069)
  chore: test renovate config [skip ci]
  chore: test renovate config [skip ci]
  chore: test renovate config [skip ci]
  chore(deps): update devdependencies (non-major) (#3846)

# Conflicts:
#	src/packages/migrate/package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants